-
Notifications
You must be signed in to change notification settings - Fork 113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: add support for custom properties for braze purchase events #2856
Conversation
Important Auto Review SkippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the To trigger a single review, invoke the TipsChat with CodeRabbit Bot (
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #2856 +/- ##
===========================================
+ Coverage 87.17% 87.19% +0.01%
===========================================
Files 772 776 +4
Lines 28805 28942 +137
Branches 6767 6788 +21
===========================================
+ Hits 25112 25235 +123
- Misses 3350 3365 +15
+ Partials 343 342 -1 ☔ View full report in Codecov by Sentry. |
Kudos, SonarCloud Quality Gate passed! |
Description of the change
Adding support for
properties
fields for Purchase events to Braze refThe
properties
field is mapped with all the extra fields perent in each of theproduct
elements present inOrder Completed
event.This change introduces additional data to be sent into Braze
Refer to linear ticket INT-1076 for further details
Type of change
Related issues
Checklists
Development
Code review